Skip to content

Conversation

@junxzm1990
Copy link
Contributor

@junxzm1990 junxzm1990 commented Nov 25, 2025

Description

This PR adds test cases for the newly introduced Common Subexpression Elimination (CSE) optimization. It consists of two groups of tests

  • optimized: test cases that are expected to be optimized
  • not-optimized: test cases that are expected to be not optimized

This is the third PR on the stack to introduce CSE.

How Has This Been Tested?

This PR adds test cases.

Expected results

  • For optimized: *.on.exp should show optimized results in file format bytecode, compared to *.off.exp.
  • For not-optimized: *.on.exp should be identical to *.off.exp in file format bytecode.

Type of Change

  • New feature
  • Bug fix
  • Breaking change
  • Performance improvement
  • Refactoring
  • Dependency update
  • Documentation update
  • Tests

Which Components or Systems Does This Change Impact?

  • Validator Node
  • Full Node (API, Indexer, etc.)
  • Move/Aptos Virtual Machine
  • Aptos Framework
  • Aptos CLI/SDK
  • Developer Infrastructure
  • Move Compiler
  • Other (specify)

Copy link
Contributor Author

Warning

This pull request is not mergeable via GitHub because a downstack PR is open. Once all requirements are satisfied, merge this PR as a stack on Graphite.
Learn more

This stack of pull requests is managed by Graphite. Learn more about stacking.

@junxzm1990 junxzm1990 force-pushed the jun/cse-opt branch 2 times, most recently from 23a4d69 to b7104d9 Compare November 25, 2025 22:51
@junxzm1990 junxzm1990 force-pushed the jun/cse-test branch 3 times, most recently from 979c25c to d9eb807 Compare November 26, 2025 01:01
@junxzm1990 junxzm1990 force-pushed the jun/cse-test branch 2 times, most recently from 1059e82 to 252c33e Compare November 26, 2025 02:21
@junxzm1990 junxzm1990 marked this pull request as ready for review November 26, 2025 16:28
@junxzm1990 junxzm1990 changed the title [move compiler] add test cases for common subexpression elimination optimizations [move compiler] [CSE Step 3] add test cases Nov 26, 2025
@junxzm1990 junxzm1990 force-pushed the jun/cse-test branch 4 times, most recently from 045d564 to 92499a6 Compare November 26, 2025 23:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants